From f8ce4ee5488b8fc075da75b442aa9f8b66ce99b4 Mon Sep 17 00:00:00 2001 From: Tim Deegan Date: Wed, 24 Nov 2010 10:20:03 +0000 Subject: [PATCH] x86/mm: remove incorrect BUG_ON. This BUG_ON tests a property of an effectively random PFN in the guest, and is explicitly _not_ seeing the MFN that's known to be owned. Signed-off-by: Tim Deegan --- xen/arch/x86/mm/p2m.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/xen/arch/x86/mm/p2m.c b/xen/arch/x86/mm/p2m.c index 04df68c515..847a59ad66 100644 --- a/xen/arch/x86/mm/p2m.c +++ b/xen/arch/x86/mm/p2m.c @@ -2380,9 +2380,6 @@ guest_physmap_add_entry(struct p2m_domain *p2m, unsigned long gfn, P2M_DEBUG("aliased! mfn=%#lx, old gfn=%#lx, new gfn=%#lx\n", mfn + i, ogfn, gfn + i); omfn = gfn_to_mfn_query(p2m, ogfn, &ot); - /* If we get here, we know the local domain owns the page, - so it can't have been grant mapped in. */ - BUG_ON( p2m_is_grant(ot) ); if ( p2m_is_ram(ot) ) { ASSERT(mfn_valid(omfn)); -- 2.30.2